Skip to content

Conversation

@letr007
Copy link
Contributor

@letr007 letr007 commented Jan 9, 2026

Modifications / 改动点

移除tool_loop_agent_runner.py中警告日志的重复的"没有"
改进注释格式

  • This is NOT a breaking change. / 这不是一个破坏性变更。

Screenshots or Test Results / 运行截图或测试结果

仅为文字性改动,对功能无影响


Checklist / 检查清单

  • 😊 如果 PR 中有新加入的功能,已经通过 Issue / 邮件等方式和作者讨论过。/ If there are new features added in the PR, I have discussed it with the authors through issues/emails, etc.
  • 👀 我的更改经过了良好的测试,并已在上方提供了“验证步骤”和“运行截图”。/ My changes have been well-tested, and "Verification Steps" and "Screenshots" have been provided above.
  • 🤓 我确保没有引入新依赖库,或者引入了新依赖库的同时将其添加到了 requirements.txtpyproject.toml 文件相应位置。/ I have ensured that no new dependencies are introduced, OR if new dependencies are introduced, they have been added to the appropriate locations in requirements.txt and pyproject.toml.
  • 😮 我的更改没有引入恶意代码。/ My changes do not introduce malicious code.

Summary by Sourcery

修复代理记录器警告消息中的重复单词,并整理附近的注释。

Bug Fixes:

  • 修正工具循环代理运行器警告日志消息中的重复单词,以提高清晰度。

Enhancements:

  • 清理工具循环代理运行器中代理循环终止路径附近注释的措辞和标点。
Original summary in English

Summary by Sourcery

Fix a duplicated word in an agent logger warning message and tidy nearby comments.

Bug Fixes:

  • Correct a duplicated word in the tool loop agent runner warning log message to improve clarity.

Enhancements:

  • Clean up comment wording and punctuation around the agent loop termination path in the tool loop agent runner.

- Fix duplicate '没有' in logger warning message
- Improve punctuation and readability of tool response comments
@auto-assign auto-assign bot requested review from Fridemn and Raven95676 January 9, 2026 11:10
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jan 9, 2026
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - 我已经审查了你的更改,看起来很棒!


Sourcery 对开源项目是免费的——如果你喜欢我们的审查,请考虑分享它们 ✨
帮我变得更有用!请在每条评论上点击 👍 或 👎,我会根据你的反馈来改进审查质量。
Original comment in English

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@dosubot dosubot bot added the area:core The bug / feature is about astrbot's core, backend label Jan 9, 2026
Copy link
Member

@Dt8333 Dt8333 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 10, 2026
@Soulter Soulter merged commit 86e9b41 into AstrBotDevs:master Jan 11, 2026
6 checks passed
@letr007 letr007 deleted the fix/tool-agent-runner-logger-typo branch January 11, 2026 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:core The bug / feature is about astrbot's core, backend lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants